-
Notifications
You must be signed in to change notification settings - Fork 18k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AP_GPS: support GPSx_GNSS_MODE for F9P #26922
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think you'll find that lots of config combinations are not allowed. We need some way to stop this trying forever
47f78b0
to
5b50d69
Compare
I have verified that all the allowed configurations work. |
cf4b743
to
aae1764
Compare
aae1764
to
514c8cd
Compare
13d200f
to
0dcd852
Compare
Hey @andyp1per is it possible to look at F10N too? |
Hi @MallikarjunSE this work was sponsored by LightDyamix and I am done at this point. There are many enhancements that could be considered - including M10 support. Happy to look at this on a commercial basis, but I would need a sample as well to get it working. Cheers Andy |
@MallikarjunSE I can test the F10N you sent me |
Thanks @tridge |
support detecting F9P hardware variant fix bug in extension buffer management support NEO-F9P GNSS configuration allow multiple configuration values to be set in one go phase F9 configuration to account for GNSS reset
0dcd852
to
9051f45
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested on F10-N and Zed-F9P
This provides limited support for the GPSx_GNSS_MODE parameter. Specifically it allows you to turn off and on the SBAS. Galileo, Beidou and Glonass constellations. All signals for these constellations are left at the defaults. GPS and QZSS are left enabled as recommended and cannot be changed.
This PR also fixes 2 bugs in the UBLOX driver:
This work kindly sponsored by LightDynamix